Skip to content

Conversation

nascheme
Copy link
Member

@nascheme nascheme commented Jan 30, 2025

This adds a new benchmark that's intended to be a better exercise for the cyclic garbage collector, compared with the existing benchmarks in the suite. The intention is to simulate an application that processes a large number of Python objects in memory, with some relatively complex set of references linking them together. The set of objects should a mixture of GC (container) and non-GC objects since that kind of mixture would be typical in many applications. I initially had the NUM_NODES parameter set to 3,000,000. However, in order to keep the running time down and reduce memory usage, I decreased it to 200,000.

I should note that this benchmark doesn't exercise the GC in terms of breaking reference cycles. The create_gc_cycles is kind of a micro benchmark of that. I'd like to add an additional benchmark that more closely simulates an application that creates a bunch of objects but only some of them are cyclic garbage. That would be separate from this PR though.

self.items[p] = extreme
elif is_big(upper_sibling):
# Case 2b.
extreme = upper_sibling.get_min_item()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this method defined?

self.items[p] = extreme
else:
# Case 2c.
extreme = upper_sibling.get_min_item()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this method defined?

# Case 2.
if is_big(node):
# Case 2a.
extreme = node.get_max_item()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this method defined?

for node_id in range(num_nodes):
a = node_id
b = f'node {node_id}'
c = rnd.randbytes(node_id % 100)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants